Skip to content

cringe-213: fix GitHub PR page content extraction#70

Merged
biancadanforth merged 2 commits into
mainfrom
cringe-213
Mar 17, 2026
Merged

cringe-213: fix GitHub PR page content extraction#70
biancadanforth merged 2 commits into
mainfrom
cringe-213

Conversation

@biancadanforth
Copy link
Copy Markdown
Collaborator

@biancadanforth biancadanforth commented Mar 17, 2026 •
edited
Loading

Because:

  • The Bugzilla links weren't showing up on GitHub PR pages due to markdown changes.

This PR:

  • Updates various methods in the content script that extract content from the page for generating Bugzilla links and adds some tests.

Related bugs:

Screenshot 2026-03-16 at 4 54 41 PM

To test:

  • Check out this branch locally
  • npx web-ext build
  • Install the extension from the build location as a temporary extension in Firefox via about:debugging
  • Open a merged GitHub Pull Request page (e.g. this one)
  • Observe/click the Bugzilla links added to the top of the page.

@biancadanforth biancadanforth requested a review from a team March 17, 2026 18:35
Haseeb702
Haseeb702 approved these changes Mar 17, 2026
View reviewed changes
Copy link
Copy Markdown

@Haseeb702 Haseeb702 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. I tested with a merged PR and all the links created by rob-bugson are functional.

@biancadanforth biancadanforth merged commit 6a21900 into main Mar 17, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Haseeb702 Haseeb702 Haseeb702 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@biancadanforth @Haseeb702